Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add log notice and CSS styles #328

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Add log notice and CSS styles #328

merged 1 commit into from
Feb 13, 2024

Conversation

ljdj
Copy link
Contributor

@ljdj ljdj commented Feb 10, 2024

Add log notice and update CSS styles

  • Implemented notifications for user actions (notice)
  • Overhauled CSS styles for UI consistency
  • Enhanced form elements' responsiveness

These changes aim to improve user experience by providing immediate feedback on performed actions and making the interface more intuitive and visually appealing.

Add log notice and update CSS styles

- Implemented notifications for user actions (notice)
- Overhauled CSS styles for UI consistency
- Enhanced form elements' responsiveness

These changes aim to improve user experience by providing immediate feedback on performed actions and making the interface more intuitive and visually appealing.
@arukompas
Copy link
Contributor

hey @ljdj , this PR contains only the Notice level. Not sure what the other 2 points in your description relate to. Did you mean to push more code? Because this PR looks good as it is - but the description confuses me a bit :)

@ljdj
Copy link
Contributor Author

ljdj commented Feb 12, 2024

Hello @arukompas,
in fact, the description of the PR is not good (AI error for the information).

Here are the changes made:

  • Added the notice level to the folder: LogLevels
  • Update of the scss (definition of a color for notice)
  • Approximate update (definition of a color for instructions)
  • Simply missing the shape of the icon to add according to your taste

I hope the explanation suits you, if the color I predefined does not suit, do not hesitate to modify.

Sincerely.

@arukompas arukompas merged commit 1218378 into opcodesio:main Feb 13, 2024
13 checks passed
@arukompas
Copy link
Contributor

@ljdj I agree with the changes, and big thanks for them!
But the AI descriptions weren't any good :) neither first nor second.

I'll tag a new release soon.

@arukompas
Copy link
Contributor

v3.2.0 tagged which now includes the new Notice level styling :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants